Skip to content

fix(vd): do not check datasourceReady for the migrating and exporting phases#2060

Open
eofff wants to merge 1 commit intomainfrom
fix/vd/do-not-require-ds-when-exporting
Open

fix(vd): do not check datasourceReady for the migrating and exporting phases#2060
eofff wants to merge 1 commit intomainfrom
fix/vd/do-not-require-ds-when-exporting

Conversation

@eofff
Copy link
Contributor

@eofff eofff commented Mar 4, 2026 •
edited
Loading

Description

Do not check datasourceReady for the migrating and exporting phases.

Checklist

  • The code is covered by unit tests.
  • e2e tests passed.
  • Documentation updated according to the changes.
  • Changes were tested in the Kubernetes cluster manually.

Changelog entries

section: vd
type: fix
summary: Do not check datasourceReady for the migrating and exporting phases.
impact_level: low

@eofff eofff added the e2e/run Run e2e test on cluster of PR author label Mar 4, 2026
@deckhouse-BOaTswain
Copy link
Contributor

deckhouse-BOaTswain commented Mar 4, 2026 •
edited
Loading

Workflow has started.
Follow the progress here: Workflow Run

The target step completed with status: failure.

@deckhouse-BOaTswain deckhouse-BOaTswain removed the e2e/run Run e2e test on cluster of PR author label Mar 4, 2026
@eofff eofff force-pushed the fix/vd/do-not-require-ds-when-exporting branch from 3143590 to 962102c Compare March 4, 2026 22:08
@eofff eofff changed the title fix(vd): do not require ds when exporting fix(vd): do not check datasourceReady for the migrating and exporting phases Mar 5, 2026
@eofff eofff added this to the v1.7.0 milestone Mar 5, 2026
@eofff eofff requested a review from LopatinDmitr March 5, 2026 08:32
@eofff eofff marked this pull request as ready for review March 5, 2026 08:32
fix
9e8aa9d 
Signed-off-by: Valeriy Khorunzhin <valeriy.khorunzhin@flant.com>
@eofff eofff force-pushed the fix/vd/do-not-require-ds-when-exporting branch from 962102c to 9e8aa9d Compare March 5, 2026 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@LopatinDmitr LopatinDmitr Awaiting requested review from LopatinDmitr

@Isteb4k Isteb4k Awaiting requested review from Isteb4k Isteb4k is a code owner

@yaroslavborbat yaroslavborbat Awaiting requested review from yaroslavborbat yaroslavborbat is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@eofff eofff

Labels

None yet

Projects

None yet

Milestone

v1.7.0

Development

Successfully merging this pull request may close these issues.

2 participants

@eofff @deckhouse-BOaTswain